Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

requirements.txt: Move instructlab to requirements-dev.txt #28

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

russellb
Copy link
Member

@russellb russellb commented Jun 19, 2024

70335a1 requirements.txt: instructlab to requirements-dev.txt

commit 70335a1
Author: Russell Bryant [email protected]
Date: Tue Jun 18 21:26:12 2024 -0400

requirements.txt: instructlab to requirements-dev.txt

This is still a dependency, but it's causing some problems in
`instructlab` repo CI. A released version of instructlab gets
installed first because of this entry, then it has to be uninstalled
before the development version can be installed.

The currently released version of `instructlab` still has schema
embedded. The next release (and current `main`) uses the
`instructlab-schema` package. Moving from one to the other in tox
is breaking the schema package.

By moving it, it will still be installed for local repo testing
purposes.

This should solve the CI failures seen here:
https://github.com/instructlab/instructlab/pull/1244

Signed-off-by: Russell Bryant <[email protected]>

This is still a dependency, but it's causing some problems in
`instructlab` repo CI. A released version of instructlab gets
installed first because of this entry, then it has to be uninstalled
before the development version can be installed.

The currently released version of `instructlab` still has schema
embedded. The next release (and current `main`) uses the
`instructlab-schema` package. Moving from one to the other in tox
is breaking the schema package.

By moving it, it will still be installed for local repo testing
purposes.

This should solve the CI failures seen here:
instructlab/instructlab#1244

Signed-off-by: Russell Bryant <[email protected]>
@russellb russellb changed the title requirements.txt: Remove instructlab requirements.txt: Move instructlab to requirements-dev.txt Jun 19, 2024
@russellb
Copy link
Member Author

This is a trivial change that's blocking the CLI change to use the library, so I'm just going to merge it.

Once we get to a working initial state, I won't do merges like this.

@russellb russellb merged commit bd941f0 into instructlab:main Jun 19, 2024
5 checks passed
jwm4 pushed a commit to jwm4/sdg that referenced this pull request Dec 13, 2024
github: Document the option of using Mergify for merge automation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant